src/interface.ts (1)

92-92: 建议添加 JSDoc 注释以改进 API 文档。

虽然类型定义正确，但建议为这个新的公共 API 属性添加 JSDoc 注释，说明其用途、默认值和行为。这将改善开发者体验和 IDE 自动完成提示。

应用此差异添加文档注释：

+  /** Enable keyboard navigation (ArrowLeft/Right for steps, Escape to close). @default false */
   keyboard?: boolean;

src/Tour.tsx (3)

168-196: 建议检查修饰键以改善用户体验。

当前实现会拦截所有 ArrowLeft/Right 按键，这可能干扰用户在输入框中使用 Ctrl+Arrow 进行单词导航等操作。建议仅在没有修饰键时处理方向键。

应用此差异添加修饰键检查：

   const handleKeyDown = React.useCallback((e: KeyboardEvent) => {
     if (!mergedOpen) {
       return;
     }
+    // 忽略带有修饰键的按键事件
+    if (e.ctrlKey || e.metaKey || e.altKey || e.shiftKey) {
+      return;
+    }
     if (e.key === 'ArrowLeft') {

---

198-205: 频繁的监听器重新注册可能影响性能。

由于 handleKeyDown 依赖于频繁变化的状态（mergedCurrent），useEffect 会在每次步骤切换时移除并重新添加监听器。虽然功能正常，但可以通过使用 ref 存储最新状态来优化。

优化方案：

+  const stateRef = React.useRef({ mergedCurrent, mergedOpen, stepsLength: steps.length });
+  
+  React.useEffect(() => {
+    stateRef.current = { mergedCurrent, mergedOpen, stepsLength: steps.length };
+  }, [mergedCurrent, mergedOpen, steps.length]);
+
-  const handleKeyDown = React.useCallback((e: KeyboardEvent) => {
-    if (!mergedOpen) {
+  const handleKeyDown = React.useCallback(() => {
+    const { mergedOpen, mergedCurrent, stepsLength } = stateRef.current;
+    if (!mergedOpen) {
       return;
     }
     // ... 其余逻辑使用 ref 中的值
-  }, [mergedCurrent, mergedOpen, steps.length, handleClose, onInternalChange]);
+  }, [handleClose, onInternalChange]);

这样监听器只在 keyboard prop 变化时重新注册。

---

61-205: 建议增强键盘导航的可访问性。

当前实现功能正常，但可以通过以下方式改善可访问性和用户体验：

1. 焦点管理：Tour 打开时，考虑将焦点移至 Tour 内容，关闭时恢复焦点到之前的元素
2. 视觉提示：在 Tour 界面上添加键盘快捷键提示（如 "使用 ← → 导航，ESC 关闭"）
3. 焦点陷阱：当 keyboard={true} 时，考虑实现焦点陷阱（focus trap），防止 Tab 键跳出 Tour 区域
4. ARIA 属性：添加 aria-keyshortcuts 属性说明可用的键盘快捷键

示例：在 Tour 内容中添加键盘提示：

// 在 TourStep 组件中添加
<div className={`${prefixCls}-keyboard-hint`} aria-live="polite">
  使用 ← → 键导航，按 ESC 关闭
</div>

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

src/Tour.tsx (3)

161-165: 考虑使用 useCallback 包裹以优化性能。

handleClose 函数逻辑正确，但未使用 useCallback 包裹，这会导致每次渲染时创建新的函数引用。由于 handleKeyDown 依赖此函数，这会导致键盘监听器在每次渲染时重新注册，可能影响性能。

应用此差异来包裹函数：

-  const handleClose = () => {
+  const handleClose = React.useCallback(() => {
     setMergedOpen(false);
     onClose?.(mergedCurrent);
-  };
+  }, [mergedCurrent, onClose]);

---

173-199: 可以使用 switch 语句提高可读性。

当前使用多个 if 语句处理不同按键。使用 switch 语句可以使代码结构更清晰、更易维护。

应用此差异重构为 switch 语句：

-    if (e.key === 'ArrowLeft') {
+    switch (e.key) {
+      case 'ArrowLeft':
-      if (mergedCurrent <= 0) {
-        return;
-      }
-      e.preventDefault();
-      e.stopPropagation();
-      onInternalChange(mergedCurrent - 1);
-      return;
-    }
-    if (e.key === 'ArrowRight') {
-      if (mergedCurrent >= steps.length - 1) {
-        return;
-      }
-      e.preventDefault();
-      e.stopPropagation();
-      onInternalChange(mergedCurrent + 1);
-      return;
-    }
-    if (e.key === 'Escape') {
-      if (!mergedClosable) {
-        return;
-      }
-      e.preventDefault();
-      e.stopPropagation();
-      handleClose();
-      return;
+        if (mergedCurrent > 0) {
+          e.preventDefault();
+          e.stopPropagation();
+          onInternalChange(mergedCurrent - 1);
+        }
+        break;
+      case 'ArrowRight':
+        if (mergedCurrent < steps.length - 1) {
+          e.preventDefault();
+          e.stopPropagation();
+          onInternalChange(mergedCurrent + 1);
+        }
+        break;
+      case 'Escape':
+        if (mergedClosable) {
+          e.preventDefault();
+          e.stopPropagation();
+          handleClose();
+        }
+        break;
+      default:
+        // 其他按键不处理
+        break;
     }

---

168-168: 优化 AbortController 的初始化。

第 168 行初始化的 AbortController 在第一次 effect 运行时立即被中止（第 203 行），从未被实际使用。可以将其初始化为 null 以避免创建不必要的对象。

应用此差异优化初始化：

-  const keyboardController = React.useRef(new AbortController());
+  const keyboardController = React.useRef<AbortController | null>(null);

然后在 effect 中添加 null 检查：

   React.useEffect(() => {
-    keyboardController.current.abort();
+    if (keyboardController.current) {
+      keyboardController.current.abort();
+    }
     keyboardController.current = new AbortController();

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro